[sklearn] Make BaseEstimator inherit from _MetadataRequester#387
[sklearn] Make BaseEstimator inherit from _MetadataRequester#387debonte merged 5 commits intomicrosoft:mainfrom
BaseEstimator inherit from _MetadataRequester#387Conversation
|
Oh, and of course, the inheritance is still there today: https://github.com/scikit-learn/scikit-learn/blob/f462edd741c3cbfccc1c6a2d64a2a66a6599d3f8/sklearn/base.py#L156 |
|
@microsoft-github-policy-service agree |
@Avasam, any idea why mypy is complaining here? I did a spot check and didn't see any explicit |
|
I'm omw out of city for the weekend. So I can't give you an exact detailed reason atm. |
|
It seems However, the type stubs already have these methods statically on the class: python-type-stubs/stubs/sklearn/utils/_metadata_requests.pyi Lines 82 to 91 in d1b4fb3 So, I think that means |
|
(I found another missing thing and added it in a new commit. Hope that's okay; I can remove it if not. It's about the global config: https://scikit-learn.org/stable/modules/generated/sklearn.set_config.html#sklearn.set_config ) |
|
Thanks for contribution @tmke8! |
This inheritance was added to sklearn two years ago here: scikit-learn/scikit-learn@62671a7#diff-ac9c452d8c660d2516089c8448596912c22417c7c690f079715593e61ef6ee0dR133